Skip to content

Conversation

@chidozieononiwu
Copy link
Member

  • This adds API-Revision view showing API Revision in detail, and ability to add APIRevisions from the review page.

Copy link
Member

@praveenkuttappan praveenkuttappan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine

@praveenkuttappan
Copy link
Member

@chidozieononiwu I see an issue. I deployed this change to staging to verify the changes. I see the comments button on left side panel but click on the conversations icon is not doing anything.

Copy link
Member

@praveenkuttappan praveenkuttappan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's the get fix to show conversations page also along with revisions. Currently conversations icon is visible but it's not doing anything when I click on it. Also what about samples icon.

@praveenkuttappan
Copy link
Member

I have also found a bug in the revisions page when adding a new revision. Newly added revision is not shown in the revision list without refreshing the whole page. We need to refresh the revisions automatically when a new revision is available. It's not a good user experience to ask user to refresh the page to get newly added revision.

@chidozieononiwu chidozieononiwu force-pushed the 071624.05/APIRevisions branch from d9e1e15 to 43b4451 Compare July 18, 2024 20:53
@chidozieononiwu chidozieononiwu self-assigned this Jul 18, 2024
@chidozieononiwu chidozieononiwu merged commit 483f731 into Azure:main Jul 18, 2024
@chidozieononiwu chidozieononiwu deleted the 071624.05/APIRevisions branch July 18, 2024 21:35
WhitShake pushed a commit to WhitShake/azure-sdk-tools that referenced this pull request Jul 24, 2024
* Add APIRevision to Review Page

* Add APIRevisions list to Review Page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants